Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Turn testing Python 2.6 in Travis on #382

Merged
merged 6 commits into from Oct 28, 2016

Conversation

jeremycline
Copy link
Member

Twisted has dropped Python 2.6 support, but fedmsg still needs to be
Python 2.6 compatible. Since moksha.hub, the dependency that pulls in
Twisted, doesn't require a specific version of Twisted, we can just
install it before we install the requirements if the environment is
Python 2.6.

Signed-off-by: Jeremy Cline jeremy@jcline.org

ashcrow and others added 5 commits October 8, 2016 16:16
Twisted has dropped Python 2.6 support, but fedmsg still needs to be
Python 2.6 compatible. Since moksha.hub, the dependency that pulls in
Twisted, doesn't require a specific version of Twisted, we can just
install it before we install the requirements if the environment is
Python 2.6.

Signed-off-by: Jeremy Cline <jeremy@jcline.org>
@jeremycline
Copy link
Member Author

This pulls in all the commits for #379 - we can either merge this and close #379, or merge #379 and rebase this. I do not mind either way.

- pip install -e .[commands,consumers]
- pip install nose mock sqlalchemy unittest2
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't unittest2 required in 2.6?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is, and it's placed in the tests_require list in setup.py

@jeremycline jeremycline force-pushed the enable-2.6-testing branch 3 times, most recently from 49629ab to 4af7adf Compare October 27, 2016 19:30
Signed-off-by: Jeremy Cline <jeremy@jcline.org>
@jeremycline
Copy link
Member Author

I added a test-requirements.txt file with all the dependencies so we can pip install them. Simply doing python setup.py nosetests to install the test requirements was causing installations to fail on 2.6 (for reasons I did not dig into too deeply).

@pypingou
Copy link
Member

👍 for me

@jeremycline jeremycline merged commit 706d494 into fedora-infra:develop Oct 28, 2016
@jeremycline jeremycline deleted the enable-2.6-testing branch October 28, 2016 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants